Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spelling #6966

Merged
merged 75 commits into from Sep 25, 2022
Merged

Spelling #6966

merged 75 commits into from Sep 25, 2022

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented May 5, 2022

Description:

This PR corrects misspellings identified by the check-spelling action.

The misspellings have been reported at jsoref@57df4bc#commitcomment-72980168

The action reports that the changes in this PR would make it happy: jsoref@905bd86

Note: this PR does not include the action. If you're interested in running a spell check on every PR and push, that can be offered separately.

BREAKING CHANGE:

Related issue (if exists):

Copy link
Contributor Author

@jsoref jsoref left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The vast majority of corrections were automatically suggested by Google Sheets.
Doubled word corrections are entirely my choice. All fault is mine.

CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
docs_app/README.md Show resolved Hide resolved
src/internal/ajax/getXHRResponse.ts Show resolved Hide resolved
// Return the subscription fron adding our subscriber to the subject.
// Return the subscription from adding our subscriber to the subject.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

front??

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@benlesh edited this in #5781. Ben, can you please help here?

src/internal/operators/repeatWhen.ts Show resolved Hide resolved
src/internal/operators/switchMap.ts Outdated Show resolved Hide resolved
src/internal/operators/timeout.ts Outdated Show resolved Hide resolved
Copy link
Member

@jakovljevic-mladen jakovljevic-mladen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jsoref, thanks a lot for such a great contribution. I'm mostly in favor of these changes, however, since I'm not a native English speaker, I'd like other core team members to review those changes as well.
I also have some suggestions, if you have time, please consider fixing them. Also, please resolve the conflicts.

CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
docs_app/README.md Show resolved Hide resolved
docs_app/src/app/app.component.ts Show resolved Hide resolved
// Return the subscription fron adding our subscriber to the subject.
// Return the subscription from adding our subscriber to the subject.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@benlesh edited this in #5781. Ben, can you please help here?

src/internal/operators/repeatWhen.ts Show resolved Hide resolved
src/internal/operators/switchMap.ts Outdated Show resolved Hide resolved
src/internal/operators/timeout.ts Outdated Show resolved Hide resolved
jsoref added 28 commits July 21, 2022 13:43
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
jsoref and others added 24 commits July 21, 2022 13:50
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Copy link
Member

@jakovljevic-mladen jakovljevic-mladen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM, however, there are some questions that @benlesh might want to look at. Thanks a lot for this PR @jsoref!

@jakovljevic-mladen
Copy link
Member

I'm not sure why I can't find an option to approve and run the pre-merge pipelines.

@benlesh benlesh merged commit 5619e0a into ReactiveX:master Sep 25, 2022
@jsoref jsoref deleted the spelling branch September 25, 2022 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants